Skip to content

adding stats aggregation docs #10251

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 9 commits into from
Jul 17, 2025
Merged

adding stats aggregation docs #10251

merged 9 commits into from
Jul 17, 2025
+237 −16

Conversation

AntonEliatra
Copy link
Contributor

Description

adding stats aggregation docs

Version

all

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@AntonEliatra
adding stats aggregation docs
822f068 
Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>
@github-actions GitHub Actions
Copy link

Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged.

Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer.

When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review.

@AntonEliatra
fixing vale errors
2b05585 
Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>
@kolchfa-aws
Copy link
Collaborator

@rishabhmaurya Could you please review this PR? Thanks!

rishabhmaurya
rishabhmaurya reviewed Jul 14, 2025
View reviewed changes
rishabhmaurya
rishabhmaurya reviewed Jul 14, 2025
View reviewed changes
@AntonEliatra
addressing PR comments
8696914 
Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>
@AntonEliatra
addressing PR comments
b98931e 
Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>
kolchfa-aws
kolchfa-aws reviewed Jul 17, 2025
View reviewed changes
Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @AntonEliatra! Please see my comments and let me know if you have any questions.

@AntonEliatra @kolchfa-aws
Apply suggestions from code review
04d6af7 
Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Signed-off-by: AntonEliatra <anton.rubin@eliatra.com>
@AntonEliatra
Copy link
Contributor Author

@kolchfa-aws thats updated now

@kolchfa-aws kolchfa-aws added Editorial review PR: Editorial review in progress and removed Tech review PR: Tech review in progress labels Jul 17, 2025
@kolchfa-aws
Copy link
Collaborator

@AntonEliatra Please fix the link checker failure. Thanks!

natebower
natebower reviewed Jul 17, 2025
View reviewed changes
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AntonEliatra Please see my comment and changes and tag me for approval when addressed. Thanks!

AntonEliatra and others added 2 commits July 17, 2025 22:25
@AntonEliatra @natebower
Apply suggestions from code review
39818fe 
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: AntonEliatra <anton.rubin@eliatra.com>
@AntonEliatra
Update stats.md
9337747 
Signed-off-by: AntonEliatra <anton.rubin@eliatra.com>
@AntonEliatra
Copy link
Contributor Author

@natebower That's updated now

@AntonEliatra
addresing PR comments
77a1eaf 
Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>
@AntonEliatra
Copy link
Contributor Author

@kolchfa-aws @natebower the links are now fixed (removed)

natebower
natebower reviewed Jul 17, 2025
View reviewed changes
@natebower
Update _aggregations/metric/stats.md
8eed10e 
Signed-off-by: Nathan Bower <nbower@amazon.com>
natebower
natebower approved these changes Jul 17, 2025
View reviewed changes
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @AntonEliatra! LGTM

@natebower natebower removed the Editorial review PR: Editorial review in progress label Jul 17, 2025
@natebower natebower merged commit f2c8530 into opensearch-project:main Jul 17, 2025
6 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 17, 2025
@github-actions @kolchfa-aws @natebower
adding stats aggregation docs (#10251)
1e6ac59 
* adding stats aggregation docs

Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>

* fixing vale errors

Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>

* addressing PR comments

Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>

* addressing PR comments

Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>

* Apply suggestions from code review

Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Signed-off-by: AntonEliatra <anton.rubin@eliatra.com>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: AntonEliatra <anton.rubin@eliatra.com>

* Update stats.md

Signed-off-by: AntonEliatra <anton.rubin@eliatra.com>

* addresing PR comments

Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>

* Update _aggregations/metric/stats.md

Signed-off-by: Nathan Bower <nbower@amazon.com>

---------

Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>
Signed-off-by: AntonEliatra <anton.rubin@eliatra.com>
Signed-off-by: Nathan Bower <nbower@amazon.com>
Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
(cherry picked from commit f2c8530)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Jul 17, 2025
Merged
github-actions bot pushed a commit that referenced this pull request Jul 17, 2025
@opensearch-trigger-bot
adding stats aggregation docs (#10251) (#10328)
9d02cb3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kolchfa-aws kolchfa-aws kolchfa-aws left review comments

@rishabhmaurya rishabhmaurya rishabhmaurya approved these changes

@natebower natebower natebower approved these changes

@AMoo-Miki AMoo-Miki Awaiting requested review from AMoo-Miki AMoo-Miki is a code owner

@dlvenable dlvenable Awaiting requested review from dlvenable dlvenable is a code owner

@epugh epugh Awaiting requested review from epugh epugh is a code owner

@sumobrian sumobrian Awaiting requested review from sumobrian sumobrian is a code owner

Assignees

@natebower natebower

@kolchfa-aws kolchfa-aws

Labels
backport 3.1 Content gap
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AntonEliatra @kolchfa-aws @rishabhmaurya @natebower